Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump plantuml and xwiki-contrib/parent-platform dependencies and fix TODOs #4

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dunklecat
Copy link

Hi all!

I've stumbled upon this repository while I was using xWiki as I needed a way to embed a pair of plantuml diagrams inside a page. I tried to minimize the edits as I only wanted to update PlantUML and the xwiki-contrib/parent-platform deps. As I dig a little inside the code I found some TODO's about problems that should be fixed by now afaik.

I admit I didn't fully tested the edits apart from what I needed and mvn test.

Let me know if there's something that I missed. Keep up the good work!

- Remove maven-javadoc-plugin dep
- Remove escapes of '\' and '/' inside PlantUMLBlockAsyncRenderer class
- Remove hardcoded URL inside TemporaryImageWriter class
@dunklecat dunklecat changed the title Bump a pair of dependencies and fix some TODOs Bump plantuml and xwiki-contrib/parent-platform dependencies and fix TODOs Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant